-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue 955 #50
Issue 955 #50
Conversation
I have little confusion about version changing. |
Please Update the README.md as well !! |
Also Have you did |
@Sarthak160 Changes i did is correct?? |
Remove the keploy-sdk from pom instead of commenting and also remove the already created testcases from the src/test folder. |
@Sarthak160 should i delete entire SamplesJavaApplication_Test.java file |
Yes , you can remove the JunitTestFile for now . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have doubt in update readme
Whats your doubt ?? |
how to change the readme file |
The issue attached to this PR has been resolved |
Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # 955
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Additional Context (Please include any Screenshots/gifs if relevant)
...
Checklist: